feat: update to RELAY spec v1.11 — remove CLI waiver, RELAY_BUILD_CLI alias#3
Merged
Conversation
…_CLI alias RELAY v1.11 §17.7 removes the waiver that allowed CLI-less C++ libraries to mark CLI requirements as not applicable. Every conformant implementation must now ship the version/capabilities/status CLI, buildable via -DRELAY_BUILD_CLI=ON. cpp-LIN already shipped the CLI in v0.1.0; this update: - Bumps kSpecVersion "1.10" -> "1.11" in relay.hpp and lin.hpp - Updates CLI JSON responses (version, capabilities) to report spec_version 1.11 - Adds RELAY_BUILD_CLI=OFF alias option to CMakeLists.txt (§17.7 compliance) - Updates all test vectors (spec_version field in relay-vectors/) - Updates REQ-RELAY-020 in .fusa-reqs.json to expect "1.11" - Sweeps v1.10 -> v1.11 across all docs: README, ROADMAP, SAFETY_PLAN, SAFETY_MANUAL, INCIDENT-RESPONSE, SECURITY, HARA, sas.md Signed-off-by: Matt Jones <matt@jellybaby.com> Signed-off-by: Matt Jones <47545907+SoundMatt@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
RELAY v1.11 §17.7 removes the waiver that previously allowed CLI-less C++ library implementations to mark CLI requirements as "not applicable". Every conformant C++ implementation must now ship the
version/capabilities/statusCLI, buildable via-DRELAY_BUILD_CLI=ON.cpp-LIN already shipped the full CLI since v0.1.0, so no new code is required — this is a version bump and compliance alignment:
kSpecVersionbumped"1.10"→"1.11"inrelay.hppandlin.hppversion,capabilities) now report"spec_version":"1.11"RELAY_BUILD_CLIalias added toCMakeLists.txtalongsideCPPLIN_BUILD_CLI(§17.7 requires-DRELAY_BUILD_CLI=ONto work)testdata/relay-vectors/) updated tospec_version: "1.11"REQ-RELAY-020updated to require"1.11"Test plan
relay conform --strict) passes with new spec_versioncpp-lin-cli versionoutputs"spec_version":"1.11"cmake -DRELAY_BUILD_CLI=ONbuilds CLI successfully (§17.7)